Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add logging for patterns and flags #432

Merged
merged 3 commits into from
Mar 14, 2022
Merged

Add logging for patterns and flags #432

merged 3 commits into from
Mar 14, 2022

Conversation

coreymckrill
Copy link
Contributor

@coreymckrill coreymckrill commented Mar 11, 2022

This enables the new logging feature in the Internal Notes plugin for the Pattern post type, and adds some custom logging for when a pattern gets flagged and when flags are managed by moderators.

This wraps up #57, albeit in a slightly different way than originally imagined. There are probably still some workflow improvements that could be made around reviewing flags, but this is an MVP, and we can iterate once we have a better idea of how pattern moderation is going to play out.

Fixes #57

Screenshot

How to test the changes in this Pull Request:

  1. Create a new pattern. Then edit the pattern.
  2. View the pattern on the front end. Click the "Report this pattern" button and submit a flag.
  3. In WP Admin, go to Block Pattern > All Flags. Mark the flag as resolved.
  4. Now view the Edit screen in WP Admin for the pattern. Open the Notes sidebar. All of your previous activity should be recorded in log entries.

@coreymckrill coreymckrill added the [Component] Pattern Directory The backend of the pattern directory: submission, management, etc label Mar 11, 2022
@coreymckrill coreymckrill self-assigned this Mar 11, 2022
@coreymckrill coreymckrill requested a review from ryelle March 11, 2022 22:17
@coreymckrill coreymckrill marked this pull request as ready for review March 11, 2022 22:17
Copy link
Contributor

@ryelle ryelle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I see all the post change logs & the flag messages 👍🏻

@coreymckrill coreymckrill merged commit 0530294 into trunk Mar 14, 2022
@coreymckrill coreymckrill deleted the fix/57/logging branch March 14, 2022 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Component] Pattern Directory The backend of the pattern directory: submission, management, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Log changes to patterns
2 participants